Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bugfix: support building docker image on openjdk23 #6984

Merged
merged 2 commits into from
Nov 6, 2024

Conversation

psxjoy
Copy link
Contributor

@psxjoy psxjoy commented Nov 6, 2024

  • I have registered the PR changes.

Ⅰ. Describe what this PR did

Support building Seata Server Docker Image on OpenJDK23.

Ⅱ. Does this pull request fix one issue?

resolve #6965

Ⅲ. Why don't you add test cases (unit test/integration test)?

Not applicable.

Ⅳ. Describe how to verify it

  1. Start with a minimal Docker image.
  2. Install Git, Maven, SDKMAN, and other necessary tools.
  3. Run the commands specified in the issue.
sdk install java 23-open
sdk use java 23-open
git clone [email protected]:apache/incubator-seata.git
cd ./incubator-seata/
./mvnw -T 4C clean package -Dimage.name=eclipse-temurin:21.0.4_7-jdk       -Pimage,release-image-based-on-java21      -DskipTests -e -Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn;
  1. when package finished ,we can see a new seata-server image.

  1. 启动一个空白的docker镜像;
  2. 安装 git, maven, sdkman 和其他工具;
  3. 按照issue中的操作执行命令;
  4. maven 构建完成,得到一个新的 seata-server 镜像。

Ⅴ. Special notes for reviews

Since there’s no key for apache/seata-server on Docker Hub, I had to verify it using my own image. You can view the built seata-server image via this link.

因为没有apache/seata-server 在dockerhub上的密钥。因此我只能通过自己的账号进行验证。可以通过这个链接 来查看构建完成的seata-server镜像。

There are other possible fixes, but I believe upgrading the version is the best approach. According to the Jib official announcement, this issue was addressed in version 3.3.0.

事实上还有其他的修复方式。但我认为升级版本是最好的解决方法。根据 jib官方公告中的宣称,它们在3.3.0版本中支持了该问题。

Copy link
Member

@xingfudeshi xingfudeshi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

codecov bot commented Nov 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.67%. Comparing base (363e179) to head (7857335).
Report is 1 commits behind head on 2.x.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x    #6984      +/-   ##
============================================
+ Coverage     52.66%   52.67%   +0.01%     
- Complexity     6571     6573       +2     
============================================
  Files          1125     1125              
  Lines         39923    39923              
  Branches       4677     4677              
============================================
+ Hits          21026    21030       +4     
+ Misses        16898    16894       -4     
  Partials       1999     1999              

see 3 files with indirect coverage changes

Copy link
Contributor

@funky-eyes funky-eyes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@funky-eyes funky-eyes added the type: bug Category issues or prs related to bug. label Nov 6, 2024
@funky-eyes funky-eyes added this to the 2.3.0 milestone Nov 6, 2024
@funky-eyes funky-eyes merged commit 78a0708 into apache:2.x Nov 6, 2024
7 checks passed
@psxjoy psxjoy deleted the docker-package-fix branch November 6, 2024 04:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Category issues or prs related to bug.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support building Seata Server Docker Image on OpenJDK23
3 participants